Skip to content

Conversation

@DavoudEshtehari
Copy link
Contributor

@DavoudEshtehari DavoudEshtehari commented Apr 14, 2022

Dropping the Microsoft.Data.SqlClient.Server namespace for supported types by Microsoft.SqlServer.Server package.

Any attempt to use the common types from Microsoft.SqlServer.Server and System.Data.SqlClient packages are known conflicts.

  • Related issue on dotnet/runtime: #66531

@DavoudEshtehari DavoudEshtehari added the Public API 🆕 Issues/PRs that introduce new APIs to the driver. label Apr 14, 2022
@DavoudEshtehari DavoudEshtehari added this to the 5.0.0-preview3 milestone Apr 14, 2022
@DavoudEshtehari
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@DavoudEshtehari DavoudEshtehari force-pushed the MSqlServerSep2 branch 2 times, most recently from a8c7f91 to 01fa0a5 Compare April 21, 2022 19:25
@DavoudEshtehari DavoudEshtehari marked this pull request as ready for review April 21, 2022 19:25
@cheenamalhotra
Copy link
Member

Hi @DavoudEshtehari

You may want to add dependency in Microsoft.Data.SqlClient.nuspec for non-netfx targets to be able to reference Microsoft.SqlServer.Server released version from external apps.

@Kaur-Parminder

This comment was marked as outdated.

# Conflicts:
#	src/Microsoft.Data.SqlClient/netcore/src/Microsoft.Data.SqlClient.csproj
#	src/Microsoft.Data.SqlClient/src/Microsoft/Data/Common/AdapterUtil.cs
@codecov
Copy link

codecov bot commented Jun 10, 2022

Codecov Report

Merging #1585 (02f02ff) into main (966fc6e) will increase coverage by 0.05%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##             main    #1585      +/-   ##
==========================================
+ Coverage   71.28%   71.34%   +0.05%     
==========================================
  Files         296      290       -6     
  Lines       61161    61048     -113     
==========================================
- Hits        43600    43554      -46     
+ Misses      17561    17494      -67     
Flag Coverage Δ
addons 92.38% <ø> (ø)
netcore 74.92% <33.33%> (+0.10%) ⬆️
netfx 69.13% <75.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...core/src/Microsoft/Data/SqlClient/SqlConnection.cs 77.19% <0.00%> (ø)
...SqlClient/src/Microsoft/Data/Common/AdapterUtil.cs 68.38% <ø> (ø)
...ient/src/Microsoft/Data/SqlClient/Server/SqlSer.cs 45.45% <ø> (ø)
...SqlClient/src/Microsoft/Data/SqlClient/SqlEnums.cs 77.37% <ø> (ø)
...lClient/src/Microsoft/Data/SqlClient/SqlUdtInfo.cs 100.00% <ø> (ø)
...etfx/src/Microsoft/Data/SqlClient/SqlConnection.cs 58.80% <75.00%> (ø)
.../netcore/src/Microsoft/Data/SqlClient/TdsParser.cs 73.70% <100.00%> (ø)
...src/Microsoft/Data/SqlClient/SqlMetadataFactory.cs 81.66% <0.00%> (-10.84%) ⬇️
...ActiveDirectoryAuthenticationTimeoutRetryHelper.cs 56.81% <0.00%> (-6.82%) ⬇️
...c/Microsoft/Data/SqlClient/SqlConnectionFactory.cs 68.85% <0.00%> (-0.82%) ⬇️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 966fc6e...02f02ff. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Public API 🆕 Issues/PRs that introduce new APIs to the driver.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants